-
-
Notifications
You must be signed in to change notification settings - Fork 43
Conversation
Travis has multiple simulator devices for iOS 10.2 (s. Issue #7031 in travis-ci)
Travis has multiple simulator devices for iOS 10.2
…atus_clean on CI,
# Conflicts: # TUM Campus App.xcodeproj/project.pbxproj
Please make this a squash and merge when done. We don't need a merge commit here as the history is kinda messy. @mammuth you can also do this locally on your branch before opening the PR: https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git |
Of course. We just squash it with the shiny new GitHub squash-before-merge-button. |
@mathiasquintero I also removed the old linked Pods_TUM_Campus_app.framework, and the second referenced group "TUM Campus App" in the last commit, since that led to errors after merging the master back into my branch. As far as I see, those are obsolete? I could imagine that I need some further fine-tuning when the merge is done. The current tests are not that meaningful right now, I will write some more later (#57) |
Just for the curious reader:
https://docs.travis-ci.com/user/pull-requests#Pull-Requests-and-Security-Restrictions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohhh. Fastlane! Fancy!
I'm going to squash and merge. Since there are some changes to the project configuration etc, I checked out this branch into a new directory and tested whether it broke something for "normal" usage (building/running the app); it seems to work. However, if one of you @mathiasquintero or @TheZoker are faced with some issues due to my changes, just drop me a line! |
Added Travis as a build server. Static Code analysis is sent to SonarQube. Some stuff will only work after this was merged into master (eg. linting new problems of PR's). Also, I will update the readme with information and instructions when the CI is up for this repo after the merge.
Closes #52